docs: add mode and mtime and .touch/.chmod mfs commands #572
Conversation
SPEC/FILES.md
Outdated
| { | ||
| path: '/tmp/myfile.txt', // The file path | ||
| content: <data> // A Buffer, Readable Stream, Pull Stream or File with the contents of the file | ||
| mode: '0755' // optional string or integer mode to store the entry with. strings will be interpreted as a base 8 number |
There was a problem hiding this comment.
When I think about a unix mode as a string I think “rwx” “r--“ etc. Why not use this format as our stringy mode?
There was a problem hiding this comment.
Yeah? I have to admit I don't when setting modes, only when viewing them with ls -l or whatever.
In my head at least, the choice was between a literal base 10 number (e.g. 493) or a literal base 8 number (e.g. 0755). Linters don't seem to like literal numbers in anything other than base 10 so that's why it's a string '0755' above.
It's good to be flexible though, maybe something we could add later?
There was a problem hiding this comment.
If it helps to get this stuff merged faster then sure 👍 We’ll be supporting rwx format in chmod so I reckon people will expect to be able to use it here too.
There was a problem hiding this comment.
We support Symbolic Notation in chmod (e.g. a=rwx, ug-x, etc) which are modifications to an existing mode.
I guess if you want to use it here you could apply the modification to the default mode for files/directories?
| await ipfs.files.chmod('/path/to/file.txt', parseInt('0777', 8)) | ||
|
|
||
| // Alternatively | ||
| await ipfs.files.chmod('/path/to/file.txt', '+rwx') |
There was a problem hiding this comment.
Not sure I follow - what does the leading ? mean in ?+rwx? I can't see any reference to it in man chmod and when I try to do this locally I get an error:
$ chmod ?+rwx foo.txt
chmod: Invalid file mode: ?+rwxThere was a problem hiding this comment.
Ah, nevermind. I didn't realize +rwx was equivalent to a+rwx.
There was a problem hiding this comment.
That's a good point, it will support u+rwx, g+r, o+w, a-w etc. though right? We should specify what subset of https://en.wikipedia.org/wiki/Modes_(Unix)#Format we do support.
There was a problem hiding this comment.
Yes, it'll support all of those.
Though now you mention it there's a testing gap for a...
..and maybe not X right now, though it can be added.
Co-Authored-By: Steven Allen <steven@stebalien.com>
|
I'm going to merge this as-is since I think we're vaguely in agreement, we can address any issues in further PRs. |
Needs more tests adding before this is ready. Early feedback is encouraged.